Skip to content

Conversation

Mat001
Copy link
Contributor

@Mat001 Mat001 commented Apr 23, 2025

Add CMAB client and unit tests.

https://jira.sso.episerver.net/browse/FSSDK-11142

@Mat001 Mat001 self-assigned this Apr 23, 2025
@Mat001 Mat001 changed the title [DRAFT][FSSDK-11142]add cmab client and tests [DRAFT][FSSDK-11142] Add cmab client and tests Apr 23, 2025
@Mat001 Mat001 changed the title [DRAFT][FSSDK-11142] Add cmab client and tests [FSSDK-11142] Add cmab client and tests Apr 29, 2025
@Mat001 Mat001 marked this pull request as ready for review April 29, 2025 22:23
@Mat001 Mat001 requested review from Copilot and raju-opti April 29, 2025 22:23
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements a new CMAB client with an integrated exponential backoff retry mechanism and unit tests for decision fetching from the CMAB API.

  • Introduces the DefaultCmabClient with retry configuration and error handling
  • Implements request marshalling, API response validation, and logging for retries

Copy link

@raju-opti raju-opti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. A few small suggestions

@Mat001 Mat001 merged commit 656583c into master May 6, 2025
20 of 21 checks passed
@Mat001 Mat001 deleted the mpirnovar-cmab-client-fssdk-11142 branch May 6, 2025 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants